Skip to content

Fix PNPM iOS Upload Debug Symbol fail in EAS Build#6086

Merged
alwx merged 6 commits intomainfrom
alwx/fix/6076
May 5, 2026
Merged

Fix PNPM iOS Upload Debug Symbol fail in EAS Build#6086
alwx merged 6 commits intomainfrom
alwx/fix/6076

Conversation

@alwx
Copy link
Copy Markdown
Contributor

@alwx alwx commented May 4, 2026

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Fixes #6076

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026 •
edited
Loading

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

  • Fix PNPM iOS Upload Debug Symbol fail in EAS Build by alwx in #6086
  • chore(deps): update Sentry Android Gradle Plugin to v6.6.0 by github-actions in #6085
  • fix(e2e): Remove flaky iOS replay assertion from captureReplay test by antonis in #6072
  • chore(deps): bump github/codeql-action from 4.35.2 to 4.35.3 by dependabot in #6078
  • fix: Prevent shell injection vulnerability in GitHub Actions workflow by fix-it-felix-sentry in #6077
  • chore(deps): update Maestro to v2.5.1 by github-actions in #6075

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026 •
edited
Loading

Fails
🚫 Pull request is not ready for merge, please add the "ready-to-merge" label to the pull request
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Fixes

- Fix PNPM iOS Upload Debug Symbol fail in EAS Build ([#6086](https://github.com/getsentry/sentry-react-native/pull/6086))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against 2ac962c

alwx
alwx commented May 4, 2026
View reviewed changes
Comment thread packages/core/scripts/sentry-xcode-debug-files.sh Outdated
@alwx alwx marked this pull request as ready for review May 4, 2026 12:52
@alwx alwx mentioned this pull request May 4, 2026
Closed
sentry-warden[bot]
sentry-warden Bot reviewed May 4, 2026
View reviewed changes
Comment thread packages/core/scripts/sentry-xcode.sh Outdated
@alwx alwx force-pushed the alwx/fix/6076 branch from 50c070f to c35ab54 Compare May 4, 2026 12:56
cursor[bot]
cursor Bot reviewed May 4, 2026
View reviewed changes
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5b2be56. Configure here.

Comment thread packages/core/scripts/sentry-xcode-debug-files.sh Outdated
antonis
antonis reviewed May 4, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the cursor feedback and the missing changelog LGTM

@alwx alwx force-pushed the alwx/fix/6076 branch from 5b2be56 to 6118dd8 Compare May 4, 2026 14:23
sentry[bot]
sentry Bot reviewed May 4, 2026
View reviewed changes
Comment thread packages/core/scripts/sentry-xcode.sh Outdated
@alwx alwx requested a review from antonis May 4, 2026 14:54
@alwx alwx force-pushed the alwx/fix/6076 branch from c7e07c6 to cef2898 Compare May 4, 2026 14:57
@alwx alwx merged commit 8929511 into main May 5, 2026
57 of 63 checks passed
@alwx alwx deleted the alwx/fix/6076 branch May 5, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentry sentry[bot] sentry[bot] left review comments

@cursor cursor[bot] cursor[bot] left review comments

@sentry-warden sentry-warden[bot] sentry-warden[bot] left review comments

@antonis antonis antonis approved these changes

@lucas-zimerman lucas-zimerman Awaiting requested review from lucas-zimerman lucas-zimerman is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

PNPM iOS Upload Debug Symbol fail in EAS Build

2 participants

@alwx @antonis